iconhelper: Move invalidation check

Just do the invalidation check once, there's no need to do it in every
branch of the switch.

Also remove useless checks: These functions will not be called if we
already have a rendered surface.

iconhelper: Improve size determination for surfaces

Instead of just working for image surface, this should now work for all
bounded surfaces.

Test included.

iconhelper: Move size computation

We do a switch on the image type, so compute the size for a certain
image type right there.

image: queue_resize() after changing icon-size property

dragsource: Store an itemdefinition, not an iconhelper

The icon is never rendered, so there's no need for the extra iconhelper
code.

imagedefinition: Split out from icon helper

The image definition is supposed to hold the description about the image
to be displayed. The icon helper actually does caching and tracks
changes.

window: Remove suspicious branch

It seems this branch is not needed anymore. It was originally added in
1999 to support gtk_widget_realize(), but all those reasons seem
obsolete today.
Instead just call gtk_widget_realize().

If you end up at this commit when bisecting:
There is no bug that made me remove this code, it was purely meant to be
cleanup / dead code removal. I seem to have introduced a new bug or
bisecting wouldn't have let you here. So it seems we should just revert
this commit.

Revert "gdk_pixbuf_get_from_window: honor device scale"

This reverts commit 657a43e54e7721fced8ef0e6808271838f14697f.

The commit breaks the assumptions about the arguments in both
gdk_pixbuf_get_from_window() and gdk_pixbuf_get_from_surface().

https://bugzilla.gnome.org/show_bug.cgi?id=757147

wayland: Remove unused variable

Poor gcc was unhappy.

popover: Make it possible to constrain to toplevel

Under X11, popovers are always constrained to the toplevel
window. Under Wayland, they aren't. This commit adds a
property that allows to explicitly constrain popovers to
the toplevel, giving them the same behavior under Wayland
as under X11.

https://bugzilla.gnome.org/show_bug.cgi?id=757474

filechooserbutton: add some spacing between icon and text

These were a bit crammed, which doesn't match well with spacing used in
various other parts of gtk.

gtkwindow: remove headerbar after disposing parent

Widgets such as gtkfilechooser may be saving their size and position on
the unmap callback, if the client-side decoration header bar is removed
first, the reported size will be wrong.

https://bugzilla.gnome.org/show_bug.cgi?id=756618

wayland: re-configure surface even if size matches

gdk-wayland backend would not re-configure a surface when its size and
scale match the known size and scale.

But there might be a pending xdg_surface_configure() that would revert
this change so we should re-configure even if the currently known
size/scale match, otherwise we may end up with a wrong size after the
xdg_surface_configure() is received.

Bugzilla: https://bugzilla.gnome.org/show_bug.cgi?id=758901

wayland: Invert drop_finished/button release call order

If we "release" the button first, the drag will be eventually cancelled,
we must first signal GDK_DROP_FINISHED, and then release the button so
the success status prevails.

wayland: Store requested target on GdkWaylandSelection

It doesn't make a lot of sense to have this stored as data offer data,
rather together with the source_targets array, which is what we're
poking here in the end.

https://bugzilla.gnome.org/show_bug.cgi?id=758713

Add some more missing nullable annotations

GtkBox: The center widget can be NULL

Propagates to GtkActionBar.

GtkWidget: Add some missing nullable annotations

inspector: Show the size request mode

This is not a property, so show it in the misc page.

window: Use g_set_object

No need to do the same thing manually.

testentryicons: update dnd example

This code was #ifdefed out since 2008. Time to try it.

testentryicons: Use an existing icon name

dialog-info is not an existing icon.

wayland: Refactor selection ownership

Dissociate ownership from our maintenance of wl_data_source objects.
The only place where ownership must be updated together is
data_source.cancelled, for the other places GDK should take care of
setting up the right ownership, even if at a different order than
we'd expect here.

This fixes GTK+ apps on wayland being locally confused about the
current selection ownership. Because gtk_selection_add_targets()
results in a wl_data_source being created, and ownership being
updated right away, early callers of this will change the ownership
even if the widget it's being called on didn't explicitly request
the selection ownership yet.

https://bugzilla.gnome.org/show_bug.cgi?id=758660

textiter: fix bug in case insensitive backward search

'win.lines' contains the same content as the GtkTextBuffer, so to find
@match_start, forward_chars_with_skipping() is called with
skip_decomp=FALSE (the last parameter). So far so good.

On the other hand, the content 'lines' (the needle split in lines) is
casefolded and normalized for a case insensitive search. So,
forward_chars_with_skipping(..., skip_decomp=TRUE) must be called only
for the portion of text containing the needle.

Since 'start_tmp' contains the location at the start of the match, we
can simply begin at that location to find the end of the match.

Unit tests are added.

https://bugzilla.gnome.org/show_bug.cgi?id=758698

listbox: Use :focus-on-click policy from row instead of list

Using the property from the container is inconsistent with the
:selectable and :activatable properties, which are per row.

icon-browser: Stop using GtkMisc properties

Instead of xpad/ypad of 4 for every widget, just increase the
row-spacing and column-spacing by 8.

https://bugzilla.gnome.org/show_bug.cgi?id=758790

icon-browser: Use proper "times" character

https://bugzilla.gnome.org/show_bug.cgi?id=758790

icon-browser: baseline-align the icon size labels

https://bugzilla.gnome.org/show_bug.cgi?id=758790

stack: Update windows before allocating children

Doing things the other way around seems to cause problems in
some cases where children want to do different things depending
on the window position.

https://bugzilla.gnome.org/show_bug.cgi?id=758563

scalebutton: Annotate the return type

Instead of just listing the return type of get_plus_button() and
get_minus_button() in the documentation, we can use the (type)
annotation to ensure that the introspection data reflects the actual
type of the returned widget.

gtkwindow: fix regression with firefox dropdown menu

Fix a regression introduced by:

commit 6866d1c widget: Make gtk_widget_queue_allocate() not resize

Where the dropdown menu in Firefox would not be relocated after the
toplevel window is moved.

bugzilla: https://bugzilla.gnome.org/show_bug.cgi?id=758609

GtkScaleButton: _get_(plus|minus)_button returns a GtkButton

https://bugzilla.gnome.org/show_bug.cgi?id=539944

Updated Portuguese translation

Updated Portuguese translation

Updated Polish translation

Adwaita: No background for spinners

Blue background for spinners doesn't make sense.

GDK W32: Add a comment for clarity

This is a copy of a similar comment in another place, which explains
why WS_EX_TRANSPARENT windows get a special treatment.

https://bugzilla.gnome.org/show_bug.cgi?id=758484

GDK W32: Clarify the use of the second argument to SetWindowPos()

https://bugzilla.gnome.org/show_bug.cgi?id=758484

GDK W32: Don't use SetWindowLong() to set/unset WS_EX_TOPMOST

While searching for the cause of bug 746745 it was discovered that one could
not set WS_EX_TOPMOST extended window style with SetWindowLong(),
but must use SetWindowPos() for that purpose.

This was never a problem most likely because it is highly unlikely for windows
to acquire/lose WS_EX_TOPMOST after they are created, by means other
than SetWindowPos() (which GTK does use to raise/lower windows and
set/remove keep_above), and because trying to set/unset WS_EX_TOPMOST with
SetWindowLong() results in WS_EX_TOPMOST merely not being set/unset (that is,
other styles are still set/unset within the same call and no error is
signalled).

https://bugzilla.gnome.org/show_bug.cgi?id=758483

wayland: Use xdg shell protocol from wayland-protocols

Drop our own copy of the xdg shell XML file, and use the one installed
by wayland-protocols.

https://bugzilla.gnome.org/show_bug.cgi?id=758634

wayland: Use pointer gestures protocol from wayland-protocols

Instead of having our own copy of the pointer gestures XML file, use
the one installed by wayland-protocols.

Since pointer gestures is an unstable protocol, it went through the
unstable protocol naming convention changes, which is reflected in this
commit.

https://bugzilla.gnome.org/show_bug.cgi?id=758634

Updated POTFILES.in

GtkWindow: Reset gestures after triggering right click titlebar action

Just like it happens for window dragging, we're likely to not see the
matching button release for this event, so we must reset the controller
manually here.

https://bugzilla.gnome.org/show_bug.cgi?id=758661

window: Don't lose position information

Before calling gdk_window_move_resize(), store the full configure
request, not just width and height.

Fixes firefox randomly losing position of its dropdown windows.

https://bugzilla.gnome.org/show_bug.cgi?id=758609

dnd: Split GtkDragSourceSite into its own file

dnd-quartz: fix missing icon helper include

dnd: Just pass iconhelper instead of whole DragSourceSite

Simplifies the code.

dnd: Remove unused struct members

iconhelper: Don't include gtk.h

iconhelper: Remove unused function

iconhelper: Remove unused context argument

Various functions don't use the style context.

wayland: remove embarrassing g_print()s

wayland: Do not store the grab cursor separately

After the grab is finished, we would expect an enter event, and
GDK updating internally the cursor for that window and device.
This means there is no need at all to store it separately in the
backend.

As a side effect, animated cursors are now also possible on grab
icons.

https://bugzilla.gnome.org/show_bug.cgi?id=735847

device: Fix dispose

The way master devices detach from their other master counterpart is
vulnerable to infinite recursion due to the way we first recurse on
the other device before clearing the pointer, this may happen if
that last reference to the other master device is held by the
device->associated field.

https://bugzilla.gnome.org/show_bug.cgi?id=732742

wayland: Plug surface leak

Other backends take care of the cairo surface destruction in
GdkWindow::destroy. We must do the same here, or the cairo_surface
and its corresponding wl_buffer are left dangling.

https://bugzilla.gnome.org/show_bug.cgi?id=747295

Updated Spanish translation

dialog: Use an element name

This will help differentiating dialogs from other windows, style-wise.

GtkPopover: Restore tails

The call to gtk_style_context_get_border was accidentally removed in
2182fe7d9d6246e7acfe4e9ef0ca26c4a6893be6.

inspector: Leave unneeded widgets out of source files

GtkWidget: Add some missing nullable annotations

GtkStack: Add some missing nullable annotations

Add a test for deprecated css properties

3.19.3

shortcuts section: Fix destroy

The container destroy implementation does not quite work here,
so do our own.

Fix some new symbols for the docs

Raleigh: Avoid deprecated icon-shadow

places view: Fix up docs

gtk: Fix make dist

Updated Spanish translation

Updated Spanish translation

searchenginetracker: Remove astray ");" closing a function

Sad face for me.

searchenginetracker: Optimize direct/recursive folder lookups

tracker:uri-is-descendant/parent has the unfortunate side effect of
rendering the collation mechanisms in the database useless, so those
require full table scans to be validated.

Performing these as pure string comparisons will perform much better,
as those allow the underlying sqlite to rely on its own collation
to perform the search, which can be significantly faster with many
elements in the database.

https://bugzilla.gnome.org/show_bug.cgi?id=758407

searchenginetracker: ensure nie:url is bound

This could produce strange warnings as it is currently passed to
tracker:uri-is-* sparql functions, as these expect no NULLs.

https://bugzilla.gnome.org/show_bug.cgi?id=758407

last last gesture stock image fixes

- swipe left was missing outline conversion to allow recolorization

Load gesture images as symbolic icons

This makes them recolor in the dark theme, which looks better.

gesture stock images size fix

fit all images to 64x64

gesture stock images fixes

- convert all objects to shapes to allow css recoloring
  ala symbolic icons

more gesture stock images tweaks

- rotation gestures more natural look

gtk-demo: Stop using GtkShortcutsGesture

Use the new predefined gestures, and GtkShortcutsShortcut.

Tweak the gtk-font-name setting docs

Clarify that only parts of the font name are used.

https://bugzilla.gnome.org/show_bug.cgi?id=758367

gesture stock images sizing

- make swipe left and right less wide

minor touchups on stock gesture images

- add minimal margin for safe scaling
- fixed height of 64pixels
- pixel alignment issues (grid fit)

shortcuts: Add set properties for optional things

This makes it possible to switch between gesture and accelerator
types and override the builtin icons and subtitles for predefined
gestures.

gtk-demo: Show all built-in gesture images

Show all the predefined gestures which have enum values
in GtkShortcutType. This also demonstrates the gesture
functionality of GtkShortcutsShortcut.

shortcuts: Add support for standard gestures

Add an enum for gestures that GtkShortcutsShortcut knows how
to show an image and subtitle for.

https://bugzilla.gnome.org/show_bug.cgi?id=758187

Add stock gesture images

These will be used in the shortcuts window.

shortcuts shortcut: Take over gesture functionality

It turns out that it is nicer in glade to have just a single
widget that can show either a shortcut or a gesture, so make
GtkShortcutsShortcut do it both.

GtkShortcutsGesture is now redundant and will be removed before
the next stable release.

shortcuts: Fix size group handling

We need to keep pointers to the size groups and apply
them to shortcuts that are added later.

shortcuts: Complete the container implementations

Various problems with the container implementations in
GtkShortcutsWindow, Section and Group were showing up
in glade.

shortcuts: Update title and section-names properly

The current code in gtkshortcutswindow.c is good enough to
construct a widget once from a .ui file, but fails to handle
changes at runtime, as happen e.g. in glade. Fix this by
listening for changes to section-name and title.

Updated POTFILES.in

Don't pass widget state flags to GtkStyleContext API

GDK W32: Only restack windows with matching always-on-top status

This prevents normal application windows (and other kinds of windows)
from being moved up in Z-order to be above windows that have the
always-on-top bit set. Doing so would make the previously-normal windows
in question also always-on-top implicitly.
Windows that are already always-on-top will be restacked on top of other
always-on-top windows too.

https://bugzilla.gnome.org/show_bug.cgi?id=746745

GDK W32: Refactor restacking functions a bit

GtkRenderBorder: make one function static

GtkCssImage: formatting fixes

gtk-demo: Fix selection in css basics demo

Updated Portuguese translation

inspector: Always underline at least one char

Empty underlines are hard to make out. Since we get somewhat
unreliable section information from the CSS parser, we just
make sure that we always underline at least one character.

cssnodedeclaration: Only set the id on the widget path if we have one

Widget paths don't like NULL.